Skip to content

fix(reviewer-bot): suppress observer for trusted pr comments#533

Merged
PLeVasseur merged 2 commits intorustfoundation:mainfrom
PLeVasseur:fix/reviewer-bot-suppress-pr-comment-observer
Mar 20, 2026
Merged

fix(reviewer-bot): suppress observer for trusted pr comments#533
PLeVasseur merged 2 commits intorustfoundation:mainfrom
PLeVasseur:fix/reviewer-bot-suppress-pr-comment-observer

Conversation

@PLeVasseur
Copy link
Copy Markdown
Collaborator

Summary

  • add a PR metadata preflight to the PR comment observer workflow so same-repo trusted human comments skip deferred artifact creation
  • keep deferred observer behavior active for cross-repo and restricted PR comment classes while avoiding duplicate direct-plus-reconcile processing for the trusted-direct class
  • add a workflow policy test covering the new observer suppression contract

Testing

  • uv run ruff check --fix .github/reviewer-bot-tests/test_reviewer_bot.py
  • uv run python -m pytest .github/reviewer-bot-tests/test_reviewer_bot.py .github/reviewer-bot-tests/test_main.py

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 20, 2026

Deploy Preview for scrc-coding-guidelines ready!

Name Link
🔨 Latest commit 23bbe58
🔍 Latest deploy log https://app.netlify.com/projects/scrc-coding-guidelines/deploys/69bdb76d5ecbc200087d59a4
😎 Deploy Preview https://deploy-preview-533--scrc-coding-guidelines.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@PLeVasseur PLeVasseur merged commit 8c895bd into rustfoundation:main Mar 20, 2026
16 checks passed
@PLeVasseur PLeVasseur deleted the fix/reviewer-bot-suppress-pr-comment-observer branch March 20, 2026 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant